Skip to content

RDKEMW-13978: Add Appgateway support for RDK8 Version#747

Open
satlead wants to merge 4 commits intodevelopfrom
RDKEMW-13978
Open

RDKEMW-13978: Add Appgateway support for RDK8 Version#747
satlead wants to merge 4 commits intodevelopfrom
RDKEMW-13978

Conversation

@satlead
Copy link
Contributor

@satlead satlead commented Feb 24, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 24, 2026 16:59
@satlead satlead requested a review from a team as a code owner February 24, 2026 16:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds version support for RDK8 by introducing a version field to two key context structures used in AppGateway and AppNotifications interfaces. This allows the system to track and use version information (potentially semantic versioning) for gateway requests and notification contexts.

Changes:

  • Added a version string field to GatewayContext struct in IAppGateway.h
  • Added a version string field to AppNotificationContext struct in IAppNotifications.h

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
apis/AppGateway/IAppGateway.h Adds version field to GatewayContext struct to track version information for gateway requests
apis/AppNotifications/IAppNotifications.h Adds version field to AppNotificationContext struct to track version information for notification contexts

ramasat
ramasat previously approved these changes Feb 24, 2026
Copilot AI review requested due to automatic review settings February 27, 2026 10:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants